Skip to content

[IMP] account: Make due date invisible for cancelled invoices #4804

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

abmn-odoo
Copy link

@abmn-odoo abmn-odoo commented Jul 2, 2025

[IMP] account: Due date is invisible for cancelled invoices

When an invoice has been canceled, the Due date for the payment is still showing and this information is not relevant anymore as no payment is expected. 

When an invoice is canceled, the Due date for the payment is invisible.

task-4908884

@abmn-odoo abmn-odoo requested a review from malb-odoo July 2, 2025 07:57
@abmn-odoo abmn-odoo self-assigned this Jul 2, 2025
@robodoo
Copy link

robodoo commented Jul 2, 2025

This PR targets the un-managed branch odoo-dev/odoo:18.0-rd-accounting-onboarding-malb, it needs to be retargeted before it can be merged.

Copy link

@malb-odoo malb-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diff looks good nicely done ! Same comments for the commit message, it's missing the task-id 👀

@malb-odoo
Copy link

Still missing the task-id there 👀

When an invoice has been canceled, the Due date for the payment is still showing and this information is not relevant anymore as no payment is expected.

When an invoice is canceled, the Due date for the payment is invisible.

task-4908884
@abmn-odoo abmn-odoo force-pushed the 18.0-rd-accounting-onboarding-invisible-due-date-for-canceled-order-abmn branch from f3c527a to 09380eb Compare July 7, 2025 07:10
@abmn-odoo abmn-odoo requested a review from malb-odoo July 7, 2025 07:10
Copy link

@malb-odoo malb-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanksss

@malb-odoo
Copy link

Since it's done and approved i will close the pr

@malb-odoo malb-odoo closed this Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants